feat: add workflow to review PRs labeled with GFI and Beginner by tech0priyanshu · Pull Request #1828 · hiero-ledger/hiero-sdk-python

tech0priyanshu · 2026-02-20T20:05:45Z

Description:

Related issue(s): need triage review for pr with GFI and beginner

Fixes #1721

Notes for reviewer:

Checklist

Documented (Code comments, README, etc.)
Tested (unit, integration, etc.)

Copilot

Pull request overview

Adds GitHub Actions automation intended to involve the triage community when PRs are labeled as beginner/GFI, and documents the change in the changelog.

Changes:

Add a new workflow that triggers on PR label events for “Good First Issue” / “Beginner”.
Add a changelog entry describing the new workflow.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File	Description
`CHANGELOG.md`	Documents the addition of a new PR triage-review workflow.
`.github/workflows/request-triage-review.yml`	Introduces a workflow meant to request triage review based on PR labels.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

coderabbitai · 2026-02-20T20:14:29Z

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

@coderabbitai resume to resume automatic reviews.
@coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

▶️ Resume reviews
🔍 Trigger review

Walkthrough

Adds a GitHub Actions workflow, a helper script, and a changelog entry to post a triage-request comment on PRs labeled "Good First Issue" or "Beginner" (or run manually). The script reads triage reviewers from a repo file or env, supports dry-run, checks idempotency, and posts comments via the GitHub API.

Changes

Cohort / File(s)	Summary
Workflow `.github/workflows/request-triage-review.yml`	New GitHub Actions workflow "Request triage review on beginner PRs": triggers on `pull_request_target` label events and `workflow_dispatch`, sets concurrency and scoped permissions, and runs a job that invokes the triage-comment script when the PR label is `Good First Issue` or `Beginner`.
Script `.github/scripts/bot-pr-gfi-review-triage.sh`	New shell script that validates environment variables, extracts the PR number from the event payload, reads triage reviewers from `.github/triage-reviewers.txt` or `TRIAGE_REVIEWERS`, enforces idempotency via a marker comment, supports dry-run, and posts a comment to request triage review using the GitHub API (`curl` + `jq`).
Changelog `CHANGELOG.md`	Adds an "Added" changelog entry documenting the new triage review automation (references issue `#1721`).

Sequence Diagram(s)

sequenceDiagram
    participant PR as "Pull Request"
    participant Workflow as "GitHub Actions\n(request-triage-review)"
    participant Script as "bot-pr-gfi-review-triage.sh"
    participant TriageList as ".github/triage-reviewers.txt\nor TRIAGE_REVIEWERS"
    participant GitHub as "GitHub API"
    participant Reviewer as "Selected triage member"

    PR->>Workflow: label added ("Good First Issue"/"Beginner") or manual dispatch
    Workflow->>Script: run script (env: GITHUB_EVENT_PATH, GITHUB_TOKEN, GITHUB_REPOSITORY, DRY_RUN)
    Script->>TriageList: read reviewers (file or env)
    Script->>GitHub: GET /repos/:owner/:repo/issues/:number/comments (check marker)
    GitHub-->>Script: comments list
    alt marker not present and not dry-run
        Script->>GitHub: POST /repos/:owner/:repo/issues/:number/comments (mention selected reviewer)
        GitHub-->>Script: 201 Created (comment id)
        Script->>Reviewer: notify via comment mention
    else marker present or dry-run
        Script-->>Workflow: no-op or preview output
    end

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	The PR implements a workflow to request triage reviews for GFI/Beginner PRs, posts comments with triage member mentions, and extracts configurable variables; however, it uses .github/triage-reviewers.txt instead of the required .github/triage-members.txt file.	Rename the triage member list file from triage-reviewers.txt to triage-members.txt as specified in issue `#1721`, or ensure the script uses the correct file path.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a workflow to review PRs labeled with GFI and Beginner.
Description check	✅ Passed	The description is related to the changeset and references the linked issue `#1721`; it documents the workflow addition for triage review requests.
Out of Scope Changes check	✅ Passed	All changes are in scope: workflow definition, triage bot script, and changelog entry directly support the issue `#1721` requirements.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

coderabbitai

Actionable comments posted: 10

coderabbitai

Actionable comments posted: 3

aceppaluni

@tech0priyanshu Thank you very much!

Was this tested on your fork?
Could we extract some of the logic into a sperate file (i.e .sh or .js)?

github-actions · 2026-02-20T20:52:36Z

Hi, this is MergeConflictBot.
Your pull request cannot be merged because it contains merge conflicts.

Please resolve these conflicts locally and push the changes.

Quick Fix for CHANGELOG.md Conflicts

If your conflict is only in CHANGELOG.md, you can resolve it easily using the GitHub web editor:

Click on the "Resolve conflicts" button in the PR
Accept both changes (keep both changelog entries)
Click "Mark as resolved"
Commit the merge

For all other merge conflicts, please read:

Thank you for contributing!

codecov · 2026-02-20T22:22:35Z

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@           Coverage Diff           @@
##             main    #1828   +/-   ##
=======================================
  Coverage   93.68%   93.68%           
=======================================
  Files         144      144           
  Lines        9350     9350           
=======================================
  Hits         8760     8760           
  Misses        590      590

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

coderabbitai

Actionable comments posted: 5

coderabbitai

Actionable comments posted: 7

coderabbitai

Actionable comments posted: 1

exploreriii

Hi @tech0priyanshu
As this workflow is not urgent, we should focus on creating clean code that is as easy as possible to maintain in the future

May i suggest you switch this from bash to javascript, using github script action
This will mean less manual constructions and is less error prone
Then please focus on writing this with clear exits and logs

Additionally it is essential to research each package you will be using, and use the latest version
docs/sdk_developers/how-to-pin-github-actions.md

We have some guides now in
docs/workflows

github-actions · 2026-02-25T10:24:26Z

Hello, this is the OfficeHourBot.

This is a reminder that the Hiero Python SDK Office Hours are scheduled in approximately 4 hours (14:00 UTC).

This session provides an opportunity to ask questions regarding this Pull Request.

Details:

Time: 14:00 UTC
Join Link: Zoom Meeting

Disclaimer: This is an automated reminder. Please verify the schedule here for any changes.

From,
The Python SDK Team

github-actions · 2026-03-03T11:10:12Z

Hi @tech0priyanshu,

This pull request has had no commit activity for 10 days. Are you still working on it?
To keep the PR active, you can:

Push a new commit.
Comment /working on the linked issue (not this PR).

If you're no longer working on this, please comment /unassign on the linked issue to release it for others. Otherwise, this PR may be closed due to inactivity.

Reach out on discord or join our office hours if you need assistance.

From the Python SDK Team

tech0priyanshu · 2026-03-07T21:52:18Z

Yeah, I will resume my work after this. I have exams from Monday to Friday, so I’m busy revising for them.

github-actions · 2026-03-17T01:54:04Z

Hi, this is WorkflowBot.
Your pull request cannot be merged as it is not passing all our workflow checks.
Please click on each check to review the logs and resolve issues so all checks pass.
To help you:

DCO signing guide
Changelog guide
Merge conflicts guide
Rebase guide
Testing guide
Discord
Community Calls
Thank you for contributing!
From the Hiero Python SDK Team

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

tech0priyanshu · 2026-04-21T10:35:20Z

tech1priyanshu/test-#1

The above repository does not have access to self-hosted runners, so I used runs-on: ubuntu-latest during testing.

Additionally, to avoid spamming PR comments, the comment is posted only once when the label is present. Subsequent runs will skip posting. During the second review, the CODEOWNERS file already assigns the relevant reviewers once they are mentioned.

cc: @exploreriii

exploreriii · 2026-04-21T16:04:54Z

Please could you also help to speed up the review process and resolve the conversations that are. Thanks

exploreriii

Requesting changes due to inactive package

codacy-production · 2026-04-21T17:04:17Z

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer}
_{TIP This summary will be updated as you push new changes. Give us feedback}

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

aceppaluni · 2026-04-22T15:44:38Z

@tech0priyanshu Can you resolve the open conversations to help speed up review?

Thank you!

Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com> fix updated things Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com> fix Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com> fixs Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com> log remove not needed Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com> update Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com> fix Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com> fixs Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com>

Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com>

tech0priyanshu · 2026-04-22T22:38:15Z

@exploreriii @aceppaluni

test -> tech1priyanshu/test-#3

exploreriii · 2026-04-24T07:24:56Z

Thanks @tech0priyanshu !

tech0priyanshu requested a review from a team as a code owner February 20, 2026 20:05

Copilot AI review requested due to automatic review settings February 20, 2026 20:05

tech0priyanshu requested a review from a team as a code owner February 20, 2026 20:05

tech0priyanshu requested review from PavelSBorisov and manishdait February 20, 2026 20:05

Copilot started reviewing on behalf of tech0priyanshu February 20, 2026 20:06 View session

Copilot AI reviewed Feb 20, 2026

View reviewed changes

coderabbitai Bot reviewed Feb 20, 2026

View reviewed changes

Comment thread .github/workflows/request-triage-review.yml Outdated

Comment thread .github/workflows/request-triage-review.yml Outdated

Comment thread .github/workflows/request-triage-review.yml Outdated

aceppaluni requested changes Feb 20, 2026

View reviewed changes

aceppaluni added the status: needs developer revision label Feb 20, 2026

tech0priyanshu force-pushed the feat/GFI_Beginner_triage branch from 0a34082 to 63cc81f Compare February 21, 2026 01:24

coderabbitai Bot reviewed Feb 21, 2026

View reviewed changes

Comment thread .github/scripts/bot-pr-gfi-review-triage.sh Outdated

tech0priyanshu marked this pull request as draft February 21, 2026 02:33

exploreriii requested changes Feb 22, 2026

View reviewed changes

exploreriii requested changes Mar 7, 2026

View reviewed changes

Comment thread .github/workflows/request-triage-review.yml

tech0priyanshu force-pushed the feat/GFI_Beginner_triage branch from f22179e to 42b6c36 Compare March 17, 2026 01:53

tech0priyanshu marked this pull request as ready for review March 17, 2026 09:13

tech0priyanshu marked this pull request as draft March 17, 2026 09:48

tech0priyanshu force-pushed the feat/GFI_Beginner_triage branch 2 times, most recently from 5079f5d to 7c2c202 Compare March 17, 2026 23:50

tech0priyanshu requested a review from Copilot April 21, 2026 10:18

Copilot started reviewing on behalf of tech0priyanshu April 21, 2026 10:18 View session

Copilot AI reviewed Apr 21, 2026

View reviewed changes

Comment thread .github/workflows/request-triage-review.yml Outdated

tech0priyanshu marked this pull request as ready for review April 21, 2026 10:26

exploreriii reviewed Apr 21, 2026

View reviewed changes

Comment thread .github/workflows/request-triage-review.yml

exploreriii requested changes Apr 21, 2026

View reviewed changes

exploreriii marked this pull request as draft April 21, 2026 16:12

tech0priyanshu force-pushed the feat/GFI_Beginner_triage branch from 5fe5551 to de3b421 Compare April 21, 2026 17:03

tech0priyanshu requested a review from Copilot April 21, 2026 17:05

Copilot started reviewing on behalf of tech0priyanshu April 21, 2026 17:06 View session

Copilot AI reviewed Apr 21, 2026

View reviewed changes

Comment thread .github/workflows/request-triage-review.yml

tech0priyanshu marked this pull request as ready for review April 21, 2026 17:26

tech0priyanshu requested a review from exploreriii April 21, 2026 17:26

exploreriii requested changes Apr 21, 2026

View reviewed changes

Comment thread .github/workflows/request-triage-review.yml Outdated

tech0priyanshu added 4 commits April 23, 2026 04:02

fix

f7dc068

Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com>

fixs

8af5a4a

Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com>

pull_target

6b30c56

Signed-off-by: tech0priyanshu <priyanshuyadv101106@gmail.com>

tech0priyanshu force-pushed the feat/GFI_Beginner_triage branch from de3b421 to 6b30c56 Compare April 22, 2026 22:35

exploreriii approved these changes Apr 23, 2026

View reviewed changes

Comment thread .github/workflows/request-triage-review.yml

exploreriii added reviewer: write requires a review with write permissions step: 2nd second stage of the review approval process labels Apr 23, 2026

aceppaluni approved these changes Apr 23, 2026

View reviewed changes

exploreriii merged commit 7d3dfd1 into hiero-ledger:main Apr 24, 2026
11 checks passed

Uh oh!

Conversation

tech0priyanshu commented Feb 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot commented Feb 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviews paused

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

❌ Failed checks (1 warning)

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

aceppaluni left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Feb 20, 2026

Quick Fix for CHANGELOG.md Conflicts

Uh oh!

codecov Bot commented Feb 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

exploreriii left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Feb 25, 2026

Uh oh!

github-actions Bot commented Mar 3, 2026

tech0priyanshu commented Feb 20, 2026 •

edited

Loading

coderabbitai Bot commented Feb 20, 2026 •

edited

Loading

codecov Bot commented Feb 20, 2026 •

edited

Loading

codacy-production Bot commented Apr 21, 2026 •

edited

Loading